Skip to content

labels in submit buttons + new screenshot #4096

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 6, 2014

Conversation

ricardclau
Copy link
Contributor

Q A
Doc fix? yes
New docs? no
Applies to 2.3+
Fixed tickets #3983

As per @weaverryan suggestion in #3983 discussion

PR is made against 2.3 branch since this is when submit buttons were added

@weaverryan
Copy link
Member

I love it. Great job!

@weaverryan weaverryan merged commit 261f6b2 into symfony:2.3 Aug 6, 2014
weaverryan added a commit that referenced this pull request Aug 6, 2014
This PR was merged into the 2.3 branch.

Discussion
----------

labels in submit buttons + new screenshot

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | 2.3+
| Fixed tickets | #3983

As per @weaverryan suggestion in #3983 discussion

PR is made against 2.3 branch since this is when submit buttons were added

Commits
-------

261f6b2 labels in submit buttons + new screenshot
@ricardclau
Copy link
Contributor Author

Thanks for merging @weaverryan

I am checking the symfony docs webpage and I am able to see my changes in the examples but not the new image. Is this happening because it uses master or is it a different process or what?

@xabbuh
Copy link
Member

xabbuh commented Aug 7, 2014

@ricardclau You can see it in the documentation for 2.3. @weaverryan didn't merge 2.3 into 2.4 and so on after your change. But he does this frequently. So, sooner or later your change will be in master too.

@ricardclau
Copy link
Contributor Author

@xabbuh When I added the comment to this PR the new image was not being displayed yet in the 2.3 docs but the code changes were. Maybe this is due to some CDN / Reverse proxy cache issue or something similar? And this is why I asked if the images may be following a different process

Anyway, glad to see it there, thanks guys :)

@javiereguiluz
Copy link
Member

@ricardclau thanks for your pull request. It's nice to have you back here :)

@xabbuh
Copy link
Member

xabbuh commented Aug 7, 2014

@ricardclau I misunderstood your question then. So yeah, I guess it's cache-related.

@ricardclau ricardclau deleted the submit_label_book branch August 7, 2014 07:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants